Skip to content

feat(guard): add policy profile dashboard#137

Merged
michiosw merged 1 commit into
mainfrom
feat/eng-340-policy-profile-ui
May 18, 2026
Merged

feat(guard): add policy profile dashboard#137
michiosw merged 1 commit into
mainfrom
feat/eng-340-policy-profile-ui

Conversation

@michiosw
Copy link
Copy Markdown
Contributor

Summary

  • explain the user-facing change
  • link the issue or context if relevant

Verification

  • go test ./...
  • go test -race ./...
  • go vet ./...
  • buf generate if protobuf or generated code changed

Release notes

  • conventional commit title matches semver intent

Copy link
Copy Markdown
Contributor Author

michiosw commented May 15, 2026

Copy link
Copy Markdown
Contributor Author

Screenshot 2026-05-16 at 13.16.48.png

@michiosw michiosw force-pushed the feat/eng-340-policy-profile-ui branch from 5f5dafc to 0ad854d Compare May 16, 2026 11:42
@michiosw michiosw force-pushed the feat/eng-339-policyconfig branch from 7e91e3c to bf88194 Compare May 16, 2026 11:42
@michiosw michiosw marked this pull request as ready for review May 16, 2026 11:44
@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented May 16, 2026

Greptile Summary

This PR adds a React dashboard for Guard policy profiles and session activity. The main changes are:

  • New policy profile API endpoints for reading and activating profiles.
  • Policy config store wiring beside the Guard sqlite database.
  • A redesigned Guard dashboard with policy cards, session activity, action groups, and an inspector panel.
  • Vite/Tailwind/TypeScript setup plus generated dashboard assets.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.

Important Files Changed

Filename Overview
internal/guard/app/server/server.go Adds policy profile routes, policy config store loading, and dashboard CORS/content-type checks.
web/guard-dashboard/src/App.tsx Adds dashboard polling, session selection, event loading, and policy activation state.
web/guard-dashboard/src/dashboard/api.ts Adds typed API parsing for sessions, events, and policy profile responses.
web/guard-dashboard/src/dashboard/PolicyPanel.tsx Adds profile cards, activation controls, metadata display, and retryable error UI.

Reviews (2): Last reviewed commit: "feat(guard): redesign dashboard UI" | Re-trigger Greptile

Comment thread internal/guard/app/server/server.go
Comment thread web/guard-dashboard/src/App.tsx
Comment thread web/guard-dashboard/src/dashboard/helpers.ts Outdated
Comment thread web/guard-dashboard/src/App.tsx
Comment thread internal/guard/app/server/server.go Outdated
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0ad854d79e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread internal/guard/app/server/server.go Outdated
@michiosw michiosw force-pushed the feat/eng-340-policy-profile-ui branch 2 times, most recently from af1d7c5 to cd47d83 Compare May 16, 2026 11:57
@michiosw michiosw force-pushed the feat/eng-339-policyconfig branch from bf88194 to bf0bed6 Compare May 16, 2026 11:57
@michiosw michiosw requested a review from hasandemirkiran May 16, 2026 11:57
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: cd47d83e2c

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread internal/guard/app/server/server.go
@michiosw michiosw force-pushed the feat/eng-340-policy-profile-ui branch 2 times, most recently from df813e0 to a4c65a3 Compare May 17, 2026 08:05
Copy link
Copy Markdown
Contributor

@greptileai

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

1 similar comment
@chatgpt-codex-connector
Copy link
Copy Markdown

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

Copy link
Copy Markdown
Contributor

@hasandemirkiran hasandemirkiran left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pushed some minor UI updates after the V2. Looks good!

Copy link
Copy Markdown
Contributor

hasandemirkiran commented May 18, 2026

Merge activity

  • May 18, 9:10 AM UTC: A user started a stack merge that includes this pull request via Graphite.
  • May 18, 9:15 AM UTC: Graphite couldn't merge this pull request because a downstack PR feat(guard): add policy config store #135 failed to merge.
  • May 18, 9:57 AM UTC: A user started a stack merge that includes this pull request via Graphite.
  • May 18, 9:58 AM UTC: Graphite rebased this pull request as part of a merge.
  • May 18, 10:00 AM UTC: @michiosw merged this pull request with Graphite.

@hasandemirkiran hasandemirkiran changed the base branch from feat/eng-339-policyconfig to graphite-base/137 May 18, 2026 09:14
@michiosw michiosw force-pushed the feat/eng-340-policy-profile-ui branch from b03bd4e to f1121ed Compare May 18, 2026 09:54
@michiosw michiosw force-pushed the graphite-base/137 branch from bf0bed6 to 6efcb1b Compare May 18, 2026 09:54
@michiosw michiosw changed the base branch from graphite-base/137 to feat/eng-339-policyconfig May 18, 2026 09:54
@michiosw michiosw changed the base branch from feat/eng-339-policyconfig to graphite-base/137 May 18, 2026 09:57
@michiosw michiosw changed the base branch from graphite-base/137 to main May 18, 2026 09:57
@michiosw michiosw dismissed hasandemirkiran’s stale review May 18, 2026 09:57

The base branch was changed.

@michiosw michiosw force-pushed the feat/eng-340-policy-profile-ui branch from f1121ed to 4c90dea Compare May 18, 2026 09:58
@michiosw michiosw merged commit 0b0e856 into main May 18, 2026
5 checks passed
@michiosw michiosw deleted the feat/eng-340-policy-profile-ui branch May 18, 2026 10:00
michiosw pushed a commit that referenced this pull request May 18, 2026
🤖 I have created a release *beep* *boop*
---


## [0.7.0](v0.6.0...v0.7.0) (2026-05-18)


### Features

* add local guard judge contract ([#132](#132)) ([ef6a3cd](ef6a3cd))
* **cli:** make kontext start local-first ([#140](#140)) ([02a34bf](02a34bf))
* **dashboard:** show guard diagnostics ([#156](#156)) ([dcbdf4a](dcbdf4a))
* **guard:** add deterministic policy engine ([#131](#131)) ([3a0f16e](3a0f16e))
* **guard:** add policy config store ([#135](#135)) ([520c3b6](520c3b6))
* **guard:** add policy profile dashboard ([#137](#137)) ([0b0e856](0b0e856))
* **guard:** connect deterministic policy and judge ([#154](#154)) ([25f0d32](25f0d32))
* manage local judge runtime ([#136](#136)) ([09f4fc0](09f4fc0))
* **runtime service:** Guard now starts a Unix-socket localruntime.Service alongside the existing HTTP daemon ([#122](#122)) ([3e87e12](3e87e12))
* **runtime service:** Introduce runtime service and make existing Unix socket more generic ([#121](#121)) ([2bbaa93](2bbaa93))


### Bug Fixes

* **dashboard:** contain command drawer text ([#157](#157)) ([4107e31](4107e31))
* **dashboard:** polish activity summary and log groups ([#153](#153)) ([fc79101](fc79101))
* harden npm dependency resolution ([#114](#114)) ([86eadf5](86eadf5))
* optimize judge fixture category matching ([#146](#146)) ([0bad7ab](0bad7ab))
* **repo:** remove repo-wide codeowners ([#133](#133)) ([73557f0](73557f0))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants